Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow the use of the right keys for one-click #3417

Merged
merged 4 commits into from
Aug 21, 2024
Merged

Conversation

Loobinex
Copy link
Member

@Loobinex Loobinex commented Aug 18, 2024

In the define keys menu, the user would press right shift/ctrl/alt to use on the one-click room commands and it would accept it as the general 'shift/control/alt', but then ingame would only accept pressing the left key. Now it accepts either.

Fixes #3415

@Loobinex Loobinex marked this pull request as ready for review August 18, 2024 22:37
@Loobinex Loobinex requested a review from eddebaby August 18, 2024 22:38
@eddebaby
Copy link
Contributor

eddebaby commented Aug 19, 2024

From a quick read of the code, I think this is a good solution - I'm having to assume some stuff so I'll have to test myself later (I'm busy this afternoon) or I'll quiz you in place of that!

Edit:
I had a bit of time so I gave it a quick test...
Notes (just reiterating what you wrote above @Loobinex):

  • Keybinding Menu unchanged (pressing Right or Left Shift/Ctrl/Alt just maps the generic Shift/Ctrl/Alt)
  • In-Game behaviour changed:
    • previously one-click would only respond to Left Shift/Ctrl/Alt to fire of the keymapping for generic Shift/Ctrl/Alt
    • with this PR, one-click responds to Right or Left Shift/Ctrl/Alt to fire of the keymapping for generic Shift/Ctrl/Alt

Bug in the PR:

  • scroll wheel is active for zoom-in/zoom-out when using Right Shift/Ctrl/Alt (scrollwheel zoom is correctly deactivated when using Left Shift/Ctrl/Alt)

There might be some other oversights too. Catch you later.

@Loobinex
Copy link
Member Author

Loobinex commented Aug 19, 2024

Bug in the PR:

* scroll wheel is active for zoom-in/zoom-out when using Right Shift/Ctrl/Alt (scrollwheel zoom is correctly deactivated when using Left Shift/Ctrl/Alt)

There might be some other oversights too. Catch you later.

Nice catch, will look into fixing this -> Should be fixed now.

@Loobinex Loobinex marked this pull request as draft August 19, 2024 23:08
@Loobinex Loobinex marked this pull request as ready for review August 19, 2024 23:08
@Loobinex Loobinex merged commit 588a7e6 into master Aug 21, 2024
@Loobinex Loobinex deleted the rightshiftoneclick branch August 21, 2024 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pair LSHIFT, LCTRL room and dig mode modifiers to RSHIFT and RCTRL
2 participants